RRFS_dev1: update clean stratigy#123
Merged
Merged
Conversation
Change the reference date for clean from current run time to current cycle time
hu5970
requested review from
CurtisAlexander-NOAA,
JeffBeck-NOAA,
TrevorAlcott-NOAA,
christinaholtNOAA,
chunhuazhou,
jaymes-kenyon and
terraladwig
as code owners
christinaholtNOAA
approved these changes
May 26, 2021
Collaborator
christinaholtNOAA
left a comment
christinaholtNOAA
left a comment
There was a problem hiding this comment.
Thanks for making the changes from back ticks to $()!
Looks reasonable to me.
| set -A XX `ls -d 20* | sort -r` | ||
| set -A XX $(ls -d 20* | sort -r) | ||
| for onetime in ${XX[*]};do | ||
| if [[ ${onetime} -le ${deletetime} ]]; then |
Collaborator
There was a problem hiding this comment.
To be more robust, suggest change line 33, 47 and similar if statements to
if [[ ${onetime} =~ '^[0-9]+$' ]] && [[ ${onetime} -le ${deletetime} ]]; then
At the RAP/HRRR era, there was one time when someone changed, for example, 20190508 to 20190508.save for debug purpose and then it crashed clean scripts for the next few days (maybe more than 10 days) , filled up the run disk very quickly.
hu5970
added a commit
that referenced
this pull request
May 29, 2021
* Feature/rrfs dev1: add retrospective function and changes for running on WCOSS (#104) * Cold start RRFS_dev1 3km CONUS hourly cycles from 09/21Z HRRR analysis. * Changes for running on RRFS cycles on WCOSS. * Copy addtional grid from UPP run directory to comout space. (#105) * Copy addtional grid from UPP run directory to comout space. * Improvements for 3-km NA (#103) * Extending walltime for "run_post" in 3-km NA * Updated settings for layout_x, layout_y to improve run time. * Increase RRFS_AK time step to match CONUS (#110) * Feature/rrfs dev1: add retro and radar tten functions (#108) * Add functions to run retro with staged data on disk. * Add namelist option for radar tten. * Bug fix for using GFS on JET for 3km NA. * Feature/rrfs dev1: delete radar tten option (#112) * delete tten option in namelist to match current USD model. * Turn off cloud analysis for future debug. (#113) * Change the "overwrite" option to "upgrade" (#111) * Change $EXPDIR backup from inside to outside (#117) * RRFS_dev1: Add python maps (#82) * Updating job script to work with maps. * Workflow for Python graphics. * Use a metatask to manage map subsets. Move skewt out of metatask. * Sorting out plotting the wgrib2-produced domains. * Updating configure scripts for appropriate domains. * Adding domains for 3DRTMA * Fixes for undefined variables and logic changes. * RRFS_dev1: Turn on radar tten option and use "upgrade" to deploy. (#121) * Add and turn on radar tten option * Change PREEXISTING_DIR_METHOD from "rename" to "upgrade" for all RRFS_dev runs. * Add configure options to decide how to clean the old cycles. (#123) Change the reference date for clean from current run time to current cycle time Co-authored-by: Jaymes Kenyon <jaymes.kenyon@noaa.gov> Co-authored-by: Trevor Alcott <69324222+TrevorAlcott-NOAA@users.noreply.github.com> Co-authored-by: Guoqing Ge <guoqing.ge@noaa.gov> Co-authored-by: Christina Holt <56881914+christinaholtNOAA@users.noreply.github.com>
Closed
Closed
Closed
Merged
This was referenced Aug 30, 2023
This was referenced Sep 25, 2023
guoqing-noaa
pushed a commit
to guoqing-noaa/regional_workflow
that referenced
this pull request
Sep 15, 2024
* Changes needed for RRFSFW run_fcst task * Fix path in config.sh_RRFSFW * Update app_build.sh for RRFSFW CCPP xml * Remove extra blank space from generate_FV3LAM_wflow.sh --------- Co-authored-by: benjamin blake <benjamin.blake@clogin05.cactus.wcoss2.ncep.noaa.gov> Co-authored-by: benjamin blake <benjamin.blake@clogin07.cactus.wcoss2.ncep.noaa.gov> Co-authored-by: benjamin blake <benjamin.blake@clogin03.cactus.wcoss2.ncep.noaa.gov>
guoqing-noaa
pushed a commit
to guoqing-noaa/regional_workflow
that referenced
this pull request
Sep 15, 2024
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
1). Add configure options to decide how to clean the old cycles.
2). Change the reference date for clean from current run time to current cycle time